Skip to content

PR by hatboysam-test #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
alicethecoder wants to merge 1 commit into
base: main
Choose a base branch
from
Open

PR by hatboysam-test #13

alicethecoder wants to merge 1 commit into from

Conversation

@alicethecoder
Copy link
Collaborator

@alicethecoder alicethecoder commented Aug 30, 2021 •
edited
Loading

Quick description

@codeapprove
Copy link

codeapprove bot commented Aug 30, 2021

👋 Review this Pull Request on CodeApprove here:
https://codeapprove.com/pr/codeapprove/meta/13

@alicethecoder alicethecoder changed the title Update index.js PR by hatboysam-test Aug 30, 2021
@codeapprove-dev
Copy link

codeapprove-dev bot commented Aug 30, 2021

👋 Review this Pull Request on CodeApprove here:
http://localhost:8080/pr/codeapprove/meta/13

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved

This looks good 👍

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval

Undo approval

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Comment

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Add comment

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Add a comment

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Comment

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Again

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

OK one more try

hatboysam
hatboysam reviewed Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Again again

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved

I approve

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Ok

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

In: Discussion
Ok

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)

In: Discussion
Draft

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

In: Discussion
Resolved

In: Discussion
Resolved

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

In: Discussion
OK

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes Aug 30, 2021
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)

In: Discussion
Ok

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (2 unresolved comments)

Undo approval

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (3 unresolved comments)

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (3 unresolved comments)

Undo approval

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (4 unresolved comments)

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Undo approval

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

This is a comment with undo approval

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

Undo with a comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Just a comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve this

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)

In: Discussion
Resolve

In: Discussion
Resolve

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Resolving discussions

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

In: Discussion
Resolve

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam previously requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Unresolved!

👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam dismissed their stale review May 28, 2022 18:32

Reviewer removed on CodeApprove

hatboysam
hatboysam requested changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

New approval

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes May 28, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

Keeping approval

In: Discussion
Resolved

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam reviewed Jun 9, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seeing if comment after approval rmoves it

hatboysam
hatboysam requested changes Jun 27, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Justnone comment

In: index.js:

> Line 6
}

Refactor

👀 @hatboysam-test it's your turn please take a look

hatboysam
hatboysam approved these changes Jun 27, 2022
View reviewed changes
Copy link
Contributor

@hatboysam hatboysam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove (go to review)

In: index.js:

> Line 6
}

Resolved

👀 @hatboysam-test it's your turn please take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hatboysam hatboysam hatboysam approved these changes

Assignees

No one assigned

Labels

codeapprove

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alicethecoder @hatboysam